docs: refresh public docs entry points#17
Conversation
|
Deploying react-native-swipe-deck with
|
| Latest commit: |
3d33264
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7b5d91bf.react-native-swipe-deck.pages.dev |
| Branch Preview URL: | https://docs-public-docs-refresh.react-native-swipe-deck.pages.dev |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3d33264210
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ```sh | ||
| yarn add @react-native-motion-kit/swipe-deck react-native-gesture-handler react-native-reanimated react-native-worklets | ||
| ``` | ||
| <PackageManagerTabs command="@react-native-motion-kit/swipe-deck react-native-gesture-handler react-native-reanimated react-native-worklets" /> |
There was a problem hiding this comment.
Rspress' PackageManagerTabs string form expects the command tail, e.g. official docs use command="install -D @rspress/core" and then rewrite install to add for yarn/pnpm/bun. Passing only package names here renders invalid install tabs such as npm @react-native-motion-kit/swipe-deck ... instead of npm install ...; the same pattern in docs/theme/index.tsx makes the homepage install command invalid too. Please include install in the string or use an explicit per-manager command object.
Useful? React with 👍 / 👎.
| ## Docs | ||
| export function ProfileDeckScreen({ profiles }: { profiles: Profile[] }) { | ||
| return ( | ||
| <View> |
There was a problem hiding this comment.
When this README example is pasted as a normal screen, the new outer View has no flex: 1 or fixed height, so the deck's own flex: 1 container can measure with height 0; getSwipeDeckState requires both layout dimensions to be positive before canSwipe becomes true, leaving the sample buttons disabled and gestures unable to commit. Add style={{ flex: 1 }} here or otherwise give the deck a definite height.
Useful? React with 👍 / 👎.
1 participant
No description provided.